From: Dylan McCall <dylan@endlessos.org>
Date: Fri, 9 Sep 2022 18:34:07 +0000 (-0700)
Subject: quick-settings: Handle interrupted menu animations
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~219^2^2~11
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=babf69f6baeaa1bba30be1d3eaf65ec8cd139355;p=gnome-shell.git

quick-settings: Handle interrupted menu animations

If the menu is closed while it is being opened, we should skip the next
animation in the chain. Similarly, if a menu is opened while it is being
closed, we should continue the animation from the previous state instead
of resetting height to 0.

Closes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/5843
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2481>
---

diff --git a/js/ui/quickSettings.js b/js/ui/quickSettings.js
index 4e9c62233..4330aa20c 100644
--- a/js/ui/quickSettings.js
+++ b/js/ui/quickSettings.js
@@ -366,6 +366,7 @@ class QuickToggleMenu extends PopupMenu.PopupMenuBase {
             reactive: true,
             x_expand: true,
             y_expand: false,
+            height: 0,
             constraints,
         });
         this.actor._delegate = this;
@@ -446,17 +447,19 @@ class QuickToggleMenu extends PopupMenu.PopupMenuBase {
         this.actor.show();
         this.isOpen = true;
 
+        const previousHeight = this.actor.height;
         this.actor.height = -1;
         const [targetHeight] = this.actor.get_preferred_height(-1);
+        this.actor.height = previousHeight;
+        const distance = Math.abs(targetHeight - previousHeight);
 
         const duration = animate !== PopupAnimation.NONE
             ? POPUP_ANIMATION_TIME / 2
             : 0;
 
-        this.actor.height = 0;
         this.box.opacity = 0;
         this.actor.ease({
-            duration,
+            duration: duration * (distance / targetHeight),
             height: targetHeight,
             onComplete: () => {
                 this.box.ease({
@@ -473,12 +476,13 @@ class QuickToggleMenu extends PopupMenu.PopupMenuBase {
         if (!this.isOpen)
             return;
 
+        const {opacity} = this.box;
         const duration = animate !== PopupAnimation.NONE
             ? POPUP_ANIMATION_TIME / 2
             : 0;
 
         this.box.ease({
-            duration,
+            duration: duration * (opacity / 255),
             opacity: 0,
             onComplete: () => {
                 this.actor.ease({